Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Adding BCD and some fixable flaws #906

Merged
merged 1 commit into from
Jan 5, 2021
Merged

Conversation

rachelandrew
Copy link
Collaborator

In mdn/sprints#3436 (comment) we decided to add the BCD for filter to the filter-function pages, so this PR does that and also fixes a couple of typos I spotted and the reported fixable flaws.

The filter page has a bunch of missing jpgs: https://developer.mozilla.org/en-US/docs/Web/CSS/filter this PR does not make that situation better or worse.

I wondered if this is the same issue @sideshowbarker encountered here: mdn/yari#2300

@rachelandrew rachelandrew requested a review from a team as a code owner January 4, 2021 09:43
@sideshowbarker
Copy link
Collaborator

The filter page has a bunch of missing jpgs: developer.mozilla.org/en-US/docs/Web/CSS/filter this PR does not make that situation better or worse.

I wondered if this is the same issue @sideshowbarker encountered here: mdn/yari#2300

I think it may be a different problem — because those images are not in-tree, because the source looks like, for example, this:

<img alt="Test_Form.jpg" id="img1" class="internal default" src="/files/3710/Test_Form_2.jpg" style="width: 100%;">

…where that /files/3710/Test_Form_2.jpg path is supposed to get redirected to this:

https://media.prod.mdn.mozit.cloud/attachments/2012/07/09/3710/7e4a42ceb861af22a38e1037e94a7ae1/Test_Form_2.jpeg

…but instead it’s going to here:

https://yari-demos.prod.mdn.mozit.cloud/files/3710/Test_Form_2.jpg

So, alerting @mdn/core-yari-dev

@peterbe
Copy link
Contributor

peterbe commented Jan 4, 2021

Ok. Let's carry on the discussion with broken images in mdn/yari#2331
It's unfortunate but unrelated.

@rachelandrew rachelandrew merged commit 17f4b04 into mdn:main Jan 5, 2021
@rachelandrew rachelandrew deleted the filter branch January 5, 2021 08:21
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Sep 5, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants